- Add chat_book_details feature flag to shared-data.ts
- Create Chat component with Salesforce Embedded Messaging integration
- Integrate Chat component into BookDetails page with feature flag
- Pass user information to Salesforce pre-chat form when available
- Configure sProduct field as "Website" for Salesforce

Related to CORE-1416

🤖 Generated with [Claude Code](https://claude.com/claude-code)

lint

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

- Add explicit hide/show logic for Salesforce chat widget
  - Hide chat elements on component unmount to prevent widget from staying visible
  - Show chat elements on component mount if widget was previously initialized
  - Targets Salesforce-injected DOM elements (.embeddedServiceHelpButton, etc.)
  - Ensures route/flag gating works correctly after initialization

- Remove setTimeout from unit test
  - Test can assert immediately since no new onload event is triggered
  - Prevents test suite slowdown and potential flakiness
  - __salesforceChatInitialized flag prevents re-init without delay

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Shorten lines

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

Comment 1: Short-circuit script loading when bootstrap already exists
- Check if window.embeddedservice_bootstrap exists on mount
- If it exists, immediately set scriptLoaded=true and skip script injection
- Prevents unnecessary script reload and speeds up widget re-display on remount
- Eliminates potential delay or failure when re-showing hidden widget

Comment 2: Only set initialization flag on successful init
- Changed initEmbeddedMessaging() to return boolean success indicator
- Only set window.__salesforceChatInitialized = true if init succeeded
- Prevents permanently broken state if initialization fails
- Allows retry on future mounts if initialization threw an error

Comment 3: Update test to reflect new script loading behavior
- Updated test comment and assertions to match short-circuit logic
- Verifies no new script is injected on remount when bootstrap exists
- Added explicit check that script count remains 0 after remount
- Clarifies that scriptLoaded is set immediately via short-circuit

Performance optimization in router.tsx:
- Extract isLoggedIn as stable primitive before useMemo
- Only re-evaluate showChat when isLoggedIn changes, not entire userContext
- Reduces unnecessary re-renders when unrelated user properties update

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Consistent return value

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

Per Roy's review comment, consolidate isLoggedIn logic into a single
canonical location in contexts/user.ts and have all other places use it.

Changes:
- Added isLoggedIn to useUserContext return value
  - Uses userStatus?.uuid || model?.uuid (the correct check)
  - Always available in context, even when userModel is not
  - Documented as the canonical way to check login state

- Updated all places that previously defined isLoggedIn locally:
  1. router.tsx - Removed local calculation, now uses context
  2. form-header.tsx - Removed incorrect last_name check, now uses context
  3. adoption.tsx - Removed incorrect last_name checks (2 instances), now uses context

Benefits:
- Single source of truth for login state
- Fixes inconsistent implementations (some used last_name incorrectly)
- Ensures all features work correctly for users without last names
- Easier to maintain and update in the future

Addresses Review 15 feedback.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Update adoption.tsx

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

1. Clear script handlers on cleanup (Comment 1)
   - Clear onload and onerror handlers before removing script
   - Prevents setState-on-unmounted-component warnings
   - Handlers can fire after unmount if network request completes late

2. Update PR description (Comments 2 & 3)
   - Clarified that widget is hidden (not removed) to preserve session state
   - Added explicit description of hide/show behavior when navigating
   - Updated User Experience section with widget visibility details
   - Updated Technical Implementation section

3. Add uuid to user context fallback branch (Comment 4)
   - Expose uuid in fallback branch: {isLoggedIn, userStatus, myOpenStaxUser, uuid: userStatus?.uuid}
   - Makes context shape consistent across both branches
   - Consumers can now reliably destructure uuid alongside isLoggedIn
   - uuid will be undefined for truly anonymous users

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>